Skip to content

feat(rate-limiting): implement dynamic concurrency adjustment based on GitHub API rate limit status#66

Merged
Einswilli merged 1 commit intomasterfrom
feature/enhanced-rate-limiting
Mar 30, 2026
Merged

feat(rate-limiting): implement dynamic concurrency adjustment based on GitHub API rate limit status#66
Einswilli merged 1 commit intomasterfrom
feature/enhanced-rate-limiting

Conversation

@Einswilli
Copy link
Copy Markdown
Contributor

Enhanced the rate limiting system to dynamically adjust download concurrency:

  • Added callback support to RateLimiter for rate limit updates
  • GitHubAPIService now invokes callbacks when rate limit info is updated
  • DownloadOrchestrator registers for rate limit callbacks and adjusts its concurrency
  • When rate limits are low, concurrency is reduced to avoid hitting limits
  • When rate limits are plentiful, concurrency returns to configured maximum
  • Maintains backward compatibility - all existing tests pass (88/88)

…n GitHub API rate limit status

Enhanced the rate limiting system to dynamically adjust download concurrency:
- Added callback support to RateLimiter for rate limit updates
- GitHubAPIService now invokes callbacks when rate limit info is updated
- DownloadOrchestrator registers for rate limit callbacks and adjusts its concurrency
- When rate limits are low, concurrency is reduced to avoid hitting limits
- When rate limits are plentiful, concurrency returns to configured maximum
- Maintains backward compatibility - all existing tests pass (88/88)
@Einswilli Einswilli merged commit d6ebbc5 into master Mar 30, 2026
1 of 2 checks passed
@Einswilli Einswilli deleted the feature/enhanced-rate-limiting branch March 30, 2026 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants